Skip to content

set required rad version to main#677

Merged
PaulHuwe merged 1 commit into
spacetelescope:mainfrom
braingram:rad_main
May 12, 2026
Merged

set required rad version to main#677
PaulHuwe merged 1 commit into
spacetelescope:mainfrom
braingram:rad_main

Conversation

@braingram

@braingram braingram commented May 11, 2026

Copy link
Copy Markdown
Contributor

spacetelescope/romancal#2313 requires that we switch the rad dependency to main to allow merging spacetelescope/rad#870 to fix the unit tests.

No regtests were run since this is just changing the rad dependency.

Tasks

  • Update or add relevant roman_datamodels tests.
  • Update relevant docstrings and / or docs/ page.
  • Does this PR change any API used downstream? (If not, label with no-changelog-entry-needed.)
News fragment change types:
  • changes/<PR#>.feature.rst: new feature
  • changes/<PR#>.bugfix.rst: fixes an issue
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.removal.rst: deprecation or removal of public API
  • changes/<PR#>.misc.rst: infrastructure or miscellaneous change

@braingram braingram marked this pull request as ready for review May 11, 2026 20:02
@braingram braingram requested a review from a team as a code owner May 11, 2026 20:02
@codecov

codecov Bot commented May 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.13%. Comparing base (087a60d) to head (088f568).
⚠️ Report is 257 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #677      +/-   ##
==========================================
- Coverage   97.56%   95.13%   -2.43%     
==========================================
  Files          30       27       -3     
  Lines        2788     2816      +28     
==========================================
- Hits         2720     2679      -41     
- Misses         68      137      +69     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@PaulHuwe PaulHuwe merged commit 1d954ab into spacetelescope:main May 12, 2026
25 of 27 checks passed
@braingram braingram deleted the rad_main branch May 12, 2026 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants